Add change-note for new SQLite models and SQLi queries #8
Conversation
Document new sqlite and sqlite_async models, the added queries to detect potential SQL injection, and associated examples and tests.
There was a problem hiding this comment.
Pull Request Overview
This PR adds a change-note documenting the introduction of new SQLite models and SQL injection detection queries for the sqlite and sqlite_async libraries, along with their associated examples and tests.
Key changes:
- Added change-note file documenting new SQLite library models
- Added SQL injection detection queries for SQLite libraries
- Included documentation updates and test coverage
| - Added models for the `sqlite` and `sqlite_async` libraries. | ||
| - Added queries to detect potential SQL injection vulnerabilities when using these libraries. | ||
| - Updated documentation to include examples and usage guidelines for the new SQLite models and queries. | ||
| - Added tests to ensure the accuracy and reliability of the new queries. No newline at end of file |
There was a problem hiding this comment.
[nitpick] Change note files should typically describe a single change or closely related set of changes. This change note combines multiple distinct changes (models, queries, documentation, and tests) which should be separated into individual change note files. Consider splitting this into separate files like 2021-01-01-new-sqlite-models.md for the models and 2021-01-01-sqlite-sqli-queries.md for the queries.
|
|
|
Changed |
| @@ -0,0 +1,7 @@ | |||
| --- | |||
| category: newQuery | |||
There was a problem hiding this comment.
The change note category should be 'minorAnalysis' rather than 'newQuery'. According to the project's change note guidelines, 'newQuery' is reserved for entirely new queries, while adding models and queries for specific libraries should use 'minorAnalysis'.
Document new sqlite and sqlite_async models, the added queries to detect potential SQL injection, and associated examples and tests.